Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

re-doing Bump_comment for version 1.4 #145

Closed
wants to merge 1 commit into from

Conversation

chazzly
Copy link
Contributor

@chazzly chazzly commented Sep 12, 2014

I was a noob and royally hosed my fork when trying to bring it in line with 1.4. So, I blew it away and started over. As such, I've closed pull request #123 and I'm re-submitting this (hopefully clean) one.

This adds an option to the spork bump plug-in to prompt the user for a comment about their change (the reason for the bump). This comment is appended to CHANGELOG.md in the following format:

<new_version>

-

The Git plugin was also updated to 'git add CHANGELOG.md' during ":after_bump"

Future ideas/ possible refinements:

  • Pass comment to a git commit.
  • Pass comment to various communications plugins (we don't currently use any of these, so may need help here).
  • Pull windows username as well as UNIX "ENV['USER']"

@chazzly chazzly changed the title re-doing change in version 1.4 re-doing Bump_comment for version 1.4 Sep 12, 2014
@jonlives jonlives added this to the Next Release milestone Nov 25, 2014
@TheSerapher
Copy link

I like this! Maybe another addition could be to automatically create a tag vX.Y.Z that can be pushed to the tree? Not sure about others, but having a tag seems mandatory nowadays to being able to revert to a previous version of a cookbook from a checkout.

At the same time, maybe check if a specific (configurable) branch is selected and if that's the case, push both that version bump, changelog change and tag into origin. We'd check for a branch to make sure we are not working within a sub-module that is disconnected from origin for whatever reason.

@chazzly
Copy link
Contributor Author

chazzly commented Dec 1, 2014

i'll take a look.

@chazzly
Copy link
Contributor Author

chazzly commented Dec 5, 2014

@jonlives, question about spork_config. As I'm working on @TheSerapher 's suggestion, and I'd like to able to set a config setting that will get picked up by another plug-in, even though i'm not directly calling it.

I've got spork-bump pulling in a couple config settings. Then, based on those, I would like to pass the comment entered to the git plug-in. Since git is not being called directly, but via the :after_bump hook, I was hoping to alter the spork_config settings for the git plug-in, but can't quite figure out how to reference it. Suggestions? It's probably something simple that I'm missing.

@jonlives
Copy link
Owner

@chazzly plugins are typically only aware of their own config, plus anything passed in via https://github.com/jonlives/knife-spork/blob/master/lib/knife-spork/runner.rb#L42 when the plugins are run.

I'm going to bump this from the 1.5.0 milestone to give you a little extra time to finish up here :)

@jonlives jonlives removed this from the Version 1.5.0 milestone Jan 26, 2015
@chazzly chazzly mentioned this pull request Feb 3, 2015
@chazzly
Copy link
Contributor Author

chazzly commented Feb 3, 2015

Closing - Superseded by #174

@chazzly chazzly closed this Feb 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants